Skip to content

Improve tests/profiler/multiple/multiple.cs - #101607

Merged
davmason merged 1 commit into
dotnet:mainfrom
jkotas:issue-100114
Apr 26, 2024
Merged

davmason merged 1 commit into
dotnet:mainfrom
jkotas:issue-100114

Conversation

@jkotas

@jkotas jkotas commented Apr 26, 2024

Copy link
Copy Markdown
Member
  • Add volatile for a field accessed by multiple threads without synchronization
  • Improve logging
  • Delete test supression against closed issue
  • Revert timeout increase that tried to work around a hang that was fixed since then

- Add volatile for a field accessed by multiple threads without
  synchronization
- Improve logging
- Delete test supression against closed issue
- Revert timeout increase that tried to work around a hang that was fixed since
  then
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

@jkotas
jkotas marked this pull request as ready for review April 26, 2024 16:09
@jkotas
jkotas requested review from davmason and tommcdon April 26, 2024 16:09
@davmason
davmason merged commit 0f562e9 into dotnet:main Apr 26, 2024
@jkotas
jkotas deleted the issue-100114 branch April 26, 2024 19:03
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
- Add volatile for a field accessed by multiple threads without
  synchronization
- Improve logging
- Delete test supression against closed issue
- Revert timeout increase that tried to work around a hang that was fixed since
  then
michaelgsharp pushed a commit to michaelgsharp/runtime that referenced this pull request May 9, 2024
- Add volatile for a field accessed by multiple threads without
  synchronization
- Improve logging
- Delete test supression against closed issue
- Revert timeout increase that tried to work around a hang that was fixed since
  then
@github-actions github-actions Bot locked and limited conversation to collaborators May 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants